Prevent Jane's scenario from being affected by import

Related issue: #968

Will Read преди 9 години
родител
ревизия
908da48286
променени са 1 файла, в които са добавени 11 реда и са изтрити 2 реда
  1. 11 2
      spec/models/scenario_import_spec.rb

+ 11 - 2
spec/models/scenario_import_spec.rb

@@ -508,7 +508,7 @@ describe ScenarioImport do
508 508
       end
509 509
     end
510 510
     
511
-    context "when an a scenario already exists with the given guid for the a different user" do
511
+    context "when Bob imports Jane's scenario" do
512 512
       let!(:existing_scenario) do
513 513
         _existing_scenerio = users(:jane).scenarios.build(:name => "an existing scenario", :description => "something")
514 514
         _existing_scenerio.guid = guid
@@ -517,11 +517,13 @@ describe ScenarioImport do
517 517
       end
518 518
       
519 519
       describe "#import" do
520
-        it "makes a new scenario" do
520
+        it "makes a new scenario for Bob" do
521 521
           expect {
522 522
             scenario_import.import(:skip_agents => true)
523 523
           }.to change { users(:bob).scenarios.count }.by(1)
524 524
 
525
+          expect(Scenario.where(guid: guid).count).to eq(2)
526
+
525 527
           expect(scenario_import.scenario.name).to eq(name)
526 528
           expect(scenario_import.scenario.description).to eq(description)
527 529
           expect(scenario_import.scenario.guid).to eq(guid)
@@ -530,6 +532,13 @@ describe ScenarioImport do
530 532
           expect(scenario_import.scenario.source_url).to eq(source_url)
531 533
           expect(scenario_import.scenario.public).to be_falsey
532 534
         end
535
+
536
+        it "does not change Jane's scenario" do
537
+          expect {
538
+            scenario_import.import(:skip_agents => true)
539
+          }.not_to change { users(:jane).scenarios }
540
+          expect(users(:jane).scenarios.find_by(guid: guid)).to eq(existing_scenario)
541
+        end
533 542
       end
534 543
     end
535 544